-
-
Notifications
You must be signed in to change notification settings - Fork 637
Require React 18 #1710
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Require React 18 #1710
Conversation
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
639df23
to
375d3a9
Compare
0d28231
to
3c5e74d
Compare
de23424
to
403d323
Compare
3c5e74d
to
fb0ac06
Compare
@alexeyr-ci2 What's the point of this PR? |
fb0ac06
to
42819f1
Compare
42819f1
to
9101887
Compare
@justin808 Added summary. |
How big is this;
Is this really worth changing now? |
@justin808 It's mostly about making our code simpler, and simplifying maintenance for the future. Obviously not right now while HiChee isn't on React 18 yet. It will be more convenient to measure bundle size change once Popmenu is on ROR 15. |
Summary
Remove React 16/17 support in order to:
This is a breaking change; do not merge until we start working on 16.0.0!
Pull Request checklist
This change is